Revert "Revert "window: Remove suspicious branch""

This reverts commit 2438a06d54636e5074c29bd696e3e81e90288b8d.

See https://bugzilla.gnome.org/show_bug.cgi?id=759705

levelbar: fix a crash

We should not try to access a block with an index that exceeds the
number of blocks in the widget.

separator toolitem: Respect ::draw property

This was lost when I converted the widget to use a gadget.

https://bugzilla.gnome.org/show_bug.cgi?id=760018

spinbutton: fix a typo

menuitem: move chaining up to draw inside gadget

This is more consistent with what other widgets do.

Updated Czech translation

testframe: Pass correct state to GtkStyleContext API

GtkImage: Simplify _set_from_icon_name

We don't need to strdup icon_name since that will happen in
_gtk_icon_helper_set_icon_name anyway.

Updated Indonesian translation
(cherry picked from commit 45b3c553a93ed6a9e3eea26e7c8337dcd9fa77b6)

Updated Indonesian translation
(cherry picked from commit 1aa5f9278423f370d7490d83e8582dae087e2f6a)

Updated Brazilian Portuguese translation
(cherry picked from commit e96c689aa1e60e561f4db67725d53fccc44234ec)

gtkdnd-quartz: fix gtk_drag_begin_internal

gtk_drag_begin_internal was changed to take a GtkImageDefinition
instead of a GtkIconHelper. This adjusts the quartz variant
accordingly.

https://bugzilla.gnome.org/show_bug.cgi?id=759607

gtkheaderbar: update window buttons also on realize

After bb34db6 _gtk_header_bar_update_window_buttons function is
called before application menu is set. This caused regression -
app menu button is initially hidden.

https://bugzilla.gnome.org/show_bug.cgi?id=756618

Updated Hebrew translation

Updated Hebrew translation

Updated Hebrew translation

Updated Hebrew translation

Fix include path for gtkiconfactoryprivate.h

Fix to compile for C libraries with no flockfile

Also use MS variant of flockfile, if available
(requires MSVCRT compatible with MSVCR90 or newer).

scrolledwindow: add missing deprecation flag

menuitem: fix a crash when reserve_indicator is TRUE

In that case, code expects an arrow gadget to be present but we're not
creating it in every occurrence.
Fix it by ensuring there will be an arrow gadget when reserve_indicator
is TRUE.

toolitem: fix a copy/paste typo

scrolledwindow: port to use a gadget

scrolledwindow: deprecate scrollbars-within-bevel style property

These days all the themes set it to TRUE, and it's not clear what
happens with overlay scrollbars...

viewport: don't render a background over the bin window

GtkViewport currently tries to draw a background over the bin window.
The feature is a bit broken at the moment, as it does not take into
account padding that might have been set on the GtkViewport, but in
general it does not seem very useful, and goes somewhat against the CSS
box model where every widget/gadget is responsible to draw its own
background. For a fix, we could either have the viewport gain a "bin"
gadget, or we could stop drawing the background.

As it isn't clear that there are any users of this feature, stop drawing
the background; a client can achieve the same effect by drawing the
background on the widget inside the viewport itself.

viewport: port to use a gadget

This will get us margin support, among other things, and simplifies the
code.

viewport: trivial code cleanup

viewport: use gtk_container_class_handle_border_width()

No need to do this manually.

gadget: Handle visibility

As GtkCssNode has the visibility concept, it makes sense to mirror it in
gadgets.

Do what visibility does in widgets: Hidden gadgets can't be drawn or
allocated and request a 0x0 size.

Note that just like widgets, gadget visibility must not be changed in
size request, allocate or draw handlers.

GtkWidget::child-visible has no equivalent yet, code will have to
emulate that manually.

Drop deprecated pango-utils api

Copy the few parsing functions we need.

Raleigh: Don't use deprecated pseudo classes

Set ID on css node right away

Previously, the ID was only set on the CSS node as a side-effect
of calling gtk_widget_get_style_context. This was showing up
in CSS style tests as nodes lacking their IDs.

Fix a comment

Add a transfer annotation

This was confusing me momentarily, so add an explicit annotation.

Updated German translation

statusicon: Avoid criticals

The code handles the possibility of pixbuf being NULL, but
after b411c318321fd4de556926119cbc6e3ba14f073a we must be
prepared for surface already being NULL. Pointed out by
Albert Muktupāvels,

https://bugzilla.gnome.org/show_bug.cgi?id=759917

checkbutton: Update state initially

Without this, the check node starts out without the :dir(ltr)
state, at least.

css: Deprecate :sorted pseudo class

This was used with regions in GtkTreeView, and is not used
anymore, so deprecate it.

Fix outline-radius css parser test

The outline-radius properties all got a -gtk prefix.

Fix the pseudo-classes-known css test

The test needs to be updated for the renamed :dnd pseudo class.
We also need to add a .errors file for the deprecation errors
that we are now producing.

css: Redo the pseudo class deprecation warnings

Putting the deprecated class behind the official variant does
not work for the case of :focus and :focused - we were matching
:focus and leave a dangling 'ed'. So, put the deprecated classes
before the official variant, and explicitly mark them as deprecated.

test-css-parser: Add a way to generate output

Use test-css-parser --generate foo.css to see the parser
output on stdout. This makes it more convenient to add new
css files to the testsuite.

Cosmetic change

inspector: Don't leak weak references

These come back to bite us when the inspector is no longer around
at the end of the program.

https://bugzilla.gnome.org/show_bug.cgi?id=759768

inspector: Remove some dead code

Since commit bffeae6203ffe5ea91988d4439571568abc95ab7, the
tree path is not used for anything useful anymore, so do away
with it entirely.

docs: Add missing value syntax for border-image

Fix a surface leak

Was introduced as part of b79a187d47ee5c45952fab60c141fcaed7e05ef6.
Results in memory leaks (and GDI objects leaks on W32).

gtkframe: format fix

gtkframe: remove unused variable

And fix the compiler warning

Redo the CSS docs

Split the CSS docs off from the GtkCssProvider docs and
give them their own chapter. Among other things, this commit
introduces more or less complete definitions of the syntax for
the supported selectors, a complete list of all supported
properties, and definitions for their values. This includes
documentation for GTK+-specific properties such as -gtk-icon-source.

Drop the :dnd syntax altogether

This was only introduced a few weeks ago. We don't need to keep
this around, now that we support the proper CSS syntax.

Adwaita: Don't use deprecated pseudo classes

Add deprecation warnings for pseudo states

Emit a deprecation error in the cases where we have
a CSS name for a custom pseudo class.

Use CSS syntax for drop highlighting

I hadn't noticed the :drop() pseudo state in the CSS4 Selectors
spec when I added this a while ago. This commit renames
GTK_STATE_FLAG_DND to GTK_STATE_FLAG_DROP_ACTIVE and adds
:drop(active) as equivalent to the :dnd pseudo state.

Add docs for GtkCssSection

GdkSeat: Use g_intern_static_string() for signal names

https://bugzilla.gnome.org/show_bug.cgi?id=759857

Updated Czech translation

Adwaita: fix notebook close button

Adwaita: use min-height/width for entry and button sizing

Adwata: fix switch slider border clash

label: Deny drag gesture when not selectable

We don't actually do anything when the label is not selectable
except for consuming the event, which breaks for instance titlebar
drags with labels that contain links. Simply deny the gesture in
that case to allow the event to bubble up normally.

https://bugzilla.gnome.org/show_bug.cgi?id=759798

gtkcssnumbervalue.c: Include fallback-c89.c to fix build

... on older Visual Studio versions, where isinf() is not available, and
copy the isinf() implementation from gdk/fallback-c89.c to
gtk/fallback-c89.c.

Forgotten files

widget: Be safe against windows losing their frame clock

gdk_widget_get_frame_clock can return NULL. In particular,
this can happen when the drag window is destroyed at the end
of a DND operation. Handle this gracefully when it happens.

gtksettings: notify after property reset

Add parser tests for animation properties

This adds tests for animation-name, animation-duration,
animation-timing-function, animation-iteration-count,
animation-direction, animation-play-state, animation-delay
and animation-fill-mode.

Add a parser tests for transition properties

This adds tests for transition-property, transition-delay
and transition-timing-function.

Add parser tests for min-width and min-height

Add parser tests for border-image properties

This adds tests for border-image-source, border-image-repeat,
border-image-slice and border-image-width.

fix printing of numbers

We should be printing infinite, not inf.

Trivial changes

menuitem: don't use content allocation to resize the GdkWindow

This is a fallout from the gadget conversion.

widget-factory: Rename outline radius in css

Rename all outline*radius properties to -gtk-outline*radius.

Adwaita: Adapt to previous commit

Rename all outline*radius properties to -gtk-outline*radius.

Prefix outline radius CSS properties

These are not in any CSS specifications, so mark them as GTK+
additions by giving them a -gtk prefix. The old names still work.

Add some docs to GtkIcon

Add some docs to GtkBuiltinIcon

stylecontext: fix gtk-doc comment syntax

Adwaita: don't set deprecated style property

Adwaita: Fix app-notification theming

The way to disable the frame border has changed.

Fix GtkFrame padding

This was showing up in the app-notification in widget-factory.

combobox: remove unused struct member

The box was removed a while ago.

Adwaita: remove combobox separator styling

The separator does not exist anymore.

combobox: deprecate shadow-type style property

It does not add any value to what can be achieved through regular CSS
styling.

combobox: port to use a gadget

Wrap the size request and allocate code into a GtkCssCustomGadget. This
makes margins work on GtkComboBox.

combobox: use a private variable when possible

Instead of dereferencing the same variable over and over.

combobox: trivial code cleanup

combobox: remove unneeded check

The combobox always has a window.

combobox: use GtkIcon to render button arrow

We can now use the newly introduced GtkIcon widget to properly render
the arrow in the combobox with CSS.

Introduce GtkIcon private class

This is an utility widget that wraps a GtkBuiltinIcon for situations
where using a real widget is more convenient than a gadget.

menuitem: Simplify separators

For regular separators, we just rely on standard CSS background
rendering nowadays. Do the same for separators in menus.

https://bugzilla.gnome.org/show_bug.cgi?id=759765

Revert "window: Remove suspicious branch"

This reverts commit 67ab00e01ec06d7b05a82c3d80b235ac6c4abfd2.

See https://bugzilla.gnome.org/show_bug.cgi?id=759705
for a case where this makes a difference.

Adwaita: fix bogus rounded corners on split headerbars

we have to do some assumptions for css selectors limits for this
particular case, so for split headerbars to work correctly the
actual haderbars need not to have the titlebar class applied.

GdkSeat: Add G_BEGIN_DECLS and G_END_DECLS

https://bugzilla.gnome.org/show_bug.cgi?id=759771

gtk/gtkbuiltinicon.c: Fix build

The gtk_builtin_icon_get_default_size_property returns a const char *,
in a way such that some compilers insist that something that is of a
pointer value be returned, so fix that by replacing 0 with NULL.

W32: avoid code duplication in gdkdevice-win32|virtual

The same get_position function is cut and pasted in the two
classes.

inspector: Avoid crash when canceling at startup

When clicking "Cancel" on the "Do you want to use GTK+ Inspector?"
dialog, unregister the update_debugging idle handler.  Also, steal
reference to 'inspector_window' while gtk_destroy_widget(), to make
further gtk_window_update_debugging() calls as a no-op.

https://bugzilla.gnome.org/show_bug.cgi?id=759764

Adwaita: Fix overwrite cursors in entries

GtkEntry uses the background color to render the text over
the block cursor. A transparent background color makes that
not work.

flowbox: remove unused prelight code

It was causing needless redraws

https://bugzilla.gnome.org/show_bug.cgi?id=759757

Fix import css parser tests

Don't use nonexisting properties here either